Skip to content

Remove features const_mut_refs and use_spin_nightly#77

Merged
phil-opp merged 2 commits into
rust-osdev:mainfrom
jannic-dev-forks:const-is-stable
Feb 26, 2023
Merged

Remove features const_mut_refs and use_spin_nightly#77
phil-opp merged 2 commits into
rust-osdev:mainfrom
jannic-dev-forks:const-is-stable

Conversation

@jannic

@jannic jannic commented Feb 24, 2023

Copy link
Copy Markdown
Contributor

Since rust 1.61, the underlying const functions are available in stable rust, and lock_api >= 0.4.7 automatically uses them. (Feature const_fn_trait_bound, rust-lang/rust#93827)

To avoid a breaking change, the features are still listed in Cargo.toml, but have no effect and are marked as deprecated. This bumps the minimum supported rust version to 1.61.

Since rust 1.61, the underlying const functions are available in
stable rust, and lock_api >= 0.4.7 automatically uses them. (Feature
const_fn_trait_bound, rust-lang/rust#93827)

To avoid a breaking change, the features are still listed in Cargo.toml,
but have no effect and are marked as deprecated.  This bumps the minimum
supported rust version to 1.61.

@phil-opp phil-opp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

Comment thread Cargo.toml Outdated
@phil-opp phil-opp enabled auto-merge February 26, 2023 11:35
@phil-opp phil-opp merged commit 07583b1 into rust-osdev:main Feb 26, 2023
@jannic jannic deleted the const-is-stable branch February 26, 2023 14:41
@jannic

jannic commented Mar 3, 2023

Copy link
Copy Markdown
Contributor Author

@phil-opp, would you consider doing a release of linked-list-allocator, so it can be used with stable rust?

@phil-opp

phil-opp commented Mar 4, 2023

Copy link
Copy Markdown
Member

Sure, published as v0.10.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants